Skip to content

Adds tests for new UiDebugOverlay features into testbed_ui #22673

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
kfc35 wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Adds tests for new UiDebugOverlay features into testbed_ui #22673

kfc35 wants to merge 4 commits into from
+165 −1

Conversation

@kfc35
Copy link
Contributor

@kfc35 kfc35 commented Jan 23, 2026 •
edited
Loading

Objective

Solution

To testbed/ui.rs, Adds a second row of entities demonstrating:

  1. border, padding, content box outlines with a rounded border radius
  2. vertical scrollbar outline
  3. horizontal scrollbar outline
  4. bidirectional scrollbar outline

The scrollbars were taken from scroll.rs and I tried to strip them down to the barest code that shows them as outlines (so scrolling doesn’t work with them)

Testing

cargo run --example testbed_ui --features="bevy_ui_debug” -- debugoutlines does the trick

Showcase

If this screenshot / these new test cases expose(s) any bugs that are not intended behavior (i.e. why is the bottom left padding box’s corner also rounded when only the bottom right border radius was specified? Or why do the bidirectional scrollbars overlap the text?), please make an issue / let me know so that I can make an issue

Screenshot Screenshot 2026-01-23 at 6 50 18 PM

@kfc35 kfc35 added A-UI Graphical user interfaces, styles, layouts, and widgets C-Testing A change that impacts how we test Bevy or how users test their apps D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward M-Deliberate-Rendering-Change An intentional change to how tests and examples are rendered labels Jan 23, 2026
@kfc35 kfc35 requested a review from ickshonpe January 23, 2026 21:41
ickshonpe
ickshonpe suggested changes Jan 23, 2026
View reviewed changes
Copy link
Contributor

@ickshonpe ickshonpe left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two tweaks I'd like:

  • The scrollbar examples are all 100 square, varying the dimensions will catch more issues.
  • The scrollbars are non-functional but you could still set non-zero ScrollPositions on some of the axes.

Otherwise, looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ickshonpe ickshonpe ickshonpe approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

A-UI Graphical user interfaces, styles, layouts, and widgets C-Testing A change that impacts how we test Bevy or how users test their apps D-Straightforward Simple bug fixes and API improvements, docs, test and examples M-Deliberate-Rendering-Change An intentional change to how tests and examples are rendered S-Needs-Review Needs reviewer attention (from anyone!) to move forward

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kfc35 @ickshonpe